Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Test server listening on IPv4/IPv6 #2206

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Test server listening on IPv4/IPv6 #2206

wants to merge 4 commits into from

Conversation

manics
Copy link
Contributor

@manics manics commented Jan 29, 2025

Describe your changes

Tests jupyter-server listen on IPv4 and IPv6 interfaces if available

Issue ticket if applicable

This is a test for #2204

Checklist (especially for first-time contributors)

  • I have performed a self-review of my code
  • If it is a core feature, I have added thorough tests
  • I will try not to use force-push to make the review process easier for reviewers
  • I have updated the documentation for significant changes

@manics
Copy link
Contributor Author

manics commented Feb 3, 2025

I've no idea why this is failing with
docker.errors.NotFound: 404 Client Error for http+docker://localhost/v1.45/networks/create: Not Found ("could not find an available, non-overlapping IPv6 address pool among the defaults to assign to the network")

This works when using docker network create ... in a minimal workflow:
https://github.com/manics/test-actions/actions/runs/13091484731/workflow

@mathbunnyru
Copy link
Member

mathbunnyru commented Feb 3, 2025 •
edited
Loading

Unfortunately, I haven't encountered this issue beofore, but I noticed it works in our self-hosted aarch64 runners.
It might be due to how GitHub-hosted aarch64 runners are configured.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@manics @mathbunnyru